Skip to content

Добавлены новые типы #155

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
theshadowco merged 1 commit into from
Jan 8, 2026
Merged

Добавлены новые типы #155

theshadowco merged 1 commit into from
Jan 8, 2026

Conversation

@theshadowco
Copy link
Member

@theshadowco theshadowco commented Jan 8, 2026 •
edited
Loading

Описание

Связанные задачи

Closes:

Чеклист

Общие

  • Ветка PR обновлена из develop
  • Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
  • Изменения покрыты тестами
  • Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Дополнительно

@coderabbitai
Copy link

coderabbitai bot commented Jan 8, 2026 •
edited
Loading

Walkthrough

Добавлены два новых enum-константы: ENUM_MANAGER в MDOValueType и BINARY_DATA в V8ValueType. Соответствующие тесты обновлены для проверки правильного количества значений в каждом enum.

Changes

Cohort / File(s) Описание изменений
MDO Value Type Enum
src/main/java/com/github/_1c_syntax/bsl/types/value/MDOValueType.java		 Добавлена новая enum-константа ENUM_MANAGER(MDOType.ENUM, "Manager", "Менеджер")
V8 Value Type Enum
src/main/java/com/github/_1c_syntax/bsl/types/value/V8ValueType.java		 Добавлена новая enum-константа BINARY_DATA("BinaryData", "ДвоичныеДанные")
MDO Value Type Tests
src/test/java/com/github/_1c_syntax/bsl/types/value/MDOValueTypeTest.java		 Обновлено ожидаемое количество значений с 57 на 58
V8 Value Type Tests
src/test/java/com/github/_1c_syntax/bsl/types/value/V8ValueTypeTest.java		 Обновлено ожидаемое количество значений с 36 на 37

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Вот Manager прыгает в enum,
А BinaryData следом вспрыгнул,
Константы множатся, как морковь в ряд,
Тесты считают — всё горит в ладах,
И числа 57, 36 изменились слегка, ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning Описание PR не содержит информации об изменениях, раздел 'Описание' пуст, а отсутствует связь с задачей (Closes: пусто) и не выполнены критические пункты чеклиста, включая precommit. Заполните раздел 'Описание' с объяснением изменений, добавьте ссылку на связанную задачу в 'Closes:' и выполните все пункты чеклиста, особенно 'gradlew precommit'.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed Название PR 'Добавлены новые типы' связано с основным изменением (добавление новых enum констант), но является слишком общим и не конкретизирует, какие именно типы добавлены.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 49fc6eb and 0adbf43.

📒 Files selected for processing (4)
  • src/main/java/com/github/_1c_syntax/bsl/types/value/MDOValueType.java
  • src/main/java/com/github/_1c_syntax/bsl/types/value/V8ValueType.java
  • src/test/java/com/github/_1c_syntax/bsl/types/value/MDOValueTypeTest.java
  • src/test/java/com/github/_1c_syntax/bsl/types/value/V8ValueTypeTest.java
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build (17, windows-latest)
🔇 Additional comments (4)
src/test/java/com/github/_1c_syntax/bsl/types/value/V8ValueTypeTest.java (1)

34-34: Тест обновлён корректно.

Ожидаемое количество значений увеличено с 36 до 37, что соответствует добавлению одной новой константы BINARY_DATA в перечисление V8ValueType.

src/test/java/com/github/_1c_syntax/bsl/types/value/MDOValueTypeTest.java (1)

53-53: Тест обновлён корректно.

Ожидаемое количество значений увеличено с 57 до 58, что соответствует добавлению одной новой константы ENUM_MANAGER в перечисление MDOValueType.

src/main/java/com/github/_1c_syntax/bsl/types/value/MDOValueType.java (1)

81-81: Константа ENUM_MANAGER добавлена корректно.

Размещение между ENUM_LIST и ENUM_REF соответствует установленному шаблону для типов метаданных. Тип менеджера для перечислений поддерживается в 1C:Enterprise 8 для чтения метаданных и работы с предопределёнными значениями.

src/main/java/com/github/_1c_syntax/bsl/types/value/V8ValueType.java (1)

43-43: Добавление константы корректно и поддерживается платформой 1С.

Константа BINARY_DATA правильно размещена в алфавитном порядке между ACCUMULATION_RECORD_TYPE и CHART, следует установленному шаблону и соответствует имеющейся нотации. Тип BinaryData/ДвоичныеДанные полностью поддерживается платформой 1C:Enterprise 8 (начиная с версии 8.3.9+) и используется для работы с бинарными данными и потоками.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 8, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@theshadowco theshadowco merged commit 043391c into develop Jan 8, 2026
16 checks passed
@theshadowco theshadowco deleted the feature/fixes260108 branch January 8, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@theshadowco